Repository navigation
This isn't really a pull request - #6
fonsecadeline wants to merge 6 commits into
Conversation
That would ensure those eligible will never be vaccinated over the year period of study, and avoid the replacing procedure (I think ?).
dc83008 to
d1acb86
Compare
If this_monday in ALL_MONDAYS but not in these_mondays then we are either before this group's Mondays, so agenda should be empty and nothing happens in replace_unvaccinated, or we are after these_mondays, and whatever we change on unvaccinated, or the pool itself, is unecessary because groups are already created.
|
On ne peut pas remplacer : for this_monday in ALL_MONDAYSpar : for this_monday in these_mondaysEn effet ces deux valeurs sont différentes.
En effet, la période d'observation des subgroups dure un an, donc une fois qu'on a épuisé tous les lundis de C'est ainsi qu'on a une boucle Étant donné que rappel de la boucle avec commentaire: # INFO: boucle sur tous les lundis:
for this_monday in ALL_MONDAYS
# INFO: création des subgroups, à partir des premiers lundis
# de chaque subgroup (pas tous les lundis!) :
if this_monday in these_mondays
subgroup = group[this_monday]
# création des vaccinés, et enregistrement de leur nombre
# dans une variable:
vaccinated_count = process_vaccinated!(
pool,
subgroup,
this_monday,
)
# création des non-vaccinés, dont le nombre est égal à celui des vaccinés:
process_first_unvaccinated!(
pool,
subgroup,
this_monday,
vaccinated_count,
agenda,
)
# INFO: `end` ! on ne créée pas de nouveaux subgroups en dehors de `these_mondays`,
# mais on doit continuer de faire des remplacements de non-vaccinés en dehors! En effet,
# la période d'observation des subgroups est plus étendue que les lundis de `this_monday`.
end
# INFO: replacement des non-vaccinés chaque lundi de `ALL_MONDAYS`. Ici, c'est bien
# tous les lundis! On ne se limite pas à `these_mondays`.
replace_unvaccinated!(
this_monday,
pool,
group,
agenda,
)
endPeut-être qu'il faut changer le nom |
2 participants
This isn't really a pull request because I could not run the code locally yet.
However it is an easier way for me to point out what I find and discuss on the code :)